Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated samples compile options #6818

Merged
merged 7 commits into from
Jul 30, 2021

Conversation

ilya-lavrenov
Copy link
Contributor

Details:

  • We don't need additional warnings for samples
  • Added CMAKE_WARN_DEPRECATED not to warn about deprecated cmake 2.8.12

@ilya-lavrenov ilya-lavrenov added category: build OpenVINO cmake script / infra category: samples OpenVINO Runtime Samples labels Jul 27, 2021
@ilya-lavrenov ilya-lavrenov added this to the 2022.1 milestone Jul 27, 2021
@ilya-lavrenov ilya-lavrenov requested review from ilyachur and a team July 27, 2021 10:27
@openvino-pushbot openvino-pushbot added category: inference OpenVINO Runtime library - Inference category: Core OpenVINO Core (aka ngraph) labels Jul 27, 2021
@ilya-lavrenov ilya-lavrenov enabled auto-merge (squash) July 30, 2021 07:24
@azhogov azhogov disabled auto-merge July 30, 2021 10:54
@azhogov azhogov merged commit 518ec79 into openvinotoolkit:master Jul 30, 2021
@ilya-lavrenov ilya-lavrenov deleted the cmake-obsolete branch July 30, 2021 10:55
rnugmanx pushed a commit to rnugmanx/openvino that referenced this pull request Aug 26, 2021
* Don't warn about obsolete cmake usage

* Removed build flags for samples

* Fixed ngraph with -Wpedantic

* cmake for 3rdparty
andrei-cv pushed a commit to andrei-cv/openvino that referenced this pull request Aug 30, 2021
* Don't warn about obsolete cmake usage

* Removed build flags for samples

* Fixed ngraph with -Wpedantic

* cmake for 3rdparty
akuporos pushed a commit to akuporos/openvino that referenced this pull request Sep 29, 2021
* Don't warn about obsolete cmake usage

* Removed build flags for samples

* Fixed ngraph with -Wpedantic

* cmake for 3rdparty
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: build OpenVINO cmake script / infra category: Core OpenVINO Core (aka ngraph) category: inference OpenVINO Runtime library - Inference category: samples OpenVINO Runtime Samples
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants